- 
                Notifications
    You must be signed in to change notification settings 
- Fork 252
CLDSRV-632 ✨ put metadata edge cases #5913
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CLDSRV-632 ✨ put metadata edge cases #5913
Conversation
| Hello darkisdude,My role is to assist you with the merge of this Available options
 Available commands
 Status report is not available. | 
| Incorrect fix versionThe  
 Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find: 
 Please check the  | 
4e1523f    to
    300aaef      
    Compare
  
    | Incorrect fix versionThe  
 Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find: 
 Please check the  | 
| Codecov Report❌ Patch coverage is  
 Additional details and impacted files
 @@                 Coverage Diff                 @@
##           development/9.1    #5913      +/-   ##
===================================================
+ Coverage            83.82%   83.83%   +0.01%     
===================================================
  Files                  193      193              
  Lines                12306    12331      +25     
===================================================
+ Hits                 10315    10338      +23     
- Misses                1991     1993       +2     
 Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
 | 
5589b9c    to
    59ee1fa      
    Compare
  
    0b5a099    to
    9c143cb      
    Compare
  
    | Incorrect fix versionThe  
 Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find: 
 Please check the  | 
| Waiting for approvalThe following approvals are needed before I can proceed with the merge: 
 | 
b5c2131    to
    da6bfb6      
    Compare
  
    e0ba601    to
    1b518c1      
    Compare
  
    1b518c1    to
    dfc7d0d      
    Compare
  
    bb7c419    to
    dfc7d0d      
    Compare
  
    | ConflictThere is a conflict between your branch  Please resolve the conflict on the feature branch ( git fetch && \
git checkout origin/feature/CLDSRV-632/put-metadata-edge-cases && \
git merge origin/development/9.1Resolve merge conflicts and commit git push origin HEAD:feature/CLDSRV-632/put-metadata-edge-cases | 
74bc07e    to
    3d7d86a      
    Compare
  
    | Waiting for approvalThe following approvals are needed before I can proceed with the merge: 
 | 
| /approve | 
| I have successfully merged the changeset of this pull request 
 The following branches have NOT changed: 
 Please check the status of the associated issue CLDSRV-632. Goodbye darkisdude. The following options are set: approve | 
Description
Reactivate some tests that were disable because some edge case were not covered.
Motivation and context
Manage 2 use case under the put metadata route:
Related issues
https://scality.atlassian.net/browse/CLDSRV-632
scality/Arsenal#2490